From 8e1d63029a5a999a65402157cda2ca503eb3866c Mon Sep 17 00:00:00 2001
From: Carlos Garnacho <carlosg@gnome.org>
Date: Mon, 24 Nov 2025 17:16:57 +0100
Subject: [PATCH] shellDBus: Add new BrightnessChanged signal to brightness
 interface

This does actually track user-triggered changes, so other pieces
(i.e. gsd-power) can react to this. This will help in reintroducing
brightness re-normalization when brightness is driven by ALS sensors.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3979>
---
 data/dbus-interfaces/org.gnome.Shell.Brightness.xml | 7 +++++++
 js/ui/shellDBus.js                                  | 9 ++++++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/data/dbus-interfaces/org.gnome.Shell.Brightness.xml b/data/dbus-interfaces/org.gnome.Shell.Brightness.xml
index a0e639728..9bfed2505 100644
--- a/data/dbus-interfaces/org.gnome.Shell.Brightness.xml
+++ b/data/dbus-interfaces/org.gnome.Shell.Brightness.xml
@@ -41,5 +41,12 @@
     -->
     <property name="HasBrightnessControl" type="b" access="read"/>
 
+    <!--
+        BrightnessChanged:
+
+        Brightness was changed by the user.
+    -->
+    <signal name="BrightnessChanged" />
+
   </interface>
 </node>
diff --git a/js/ui/shellDBus.js b/js/ui/shellDBus.js
index 17f43e46e..1e5a6a322 100644
--- a/js/ui/shellDBus.js
+++ b/js/ui/shellDBus.js
@@ -602,7 +602,10 @@ export class BrightnessDBus {
         Gio.DBus.session.own_name('org.gnome.Shell.Brightness',
             Gio.BusNameOwnerFlags.NONE, null, null);
 
-        this._manager.connectObject('changed', this._sync.bind(this), this);
+        this._manager.connectObject(
+            'changed', () => this._sync(),
+            'user-update', () => this._userChange(),
+            this);
         this._sync();
     }
 
@@ -616,6 +619,10 @@ export class BrightnessDBus {
             new GLib.Variant('b', this._hasBrightnessControl));
     }
 
+    _userChange() {
+        this._dbusImpl.emit_signal('BrightnessChanged', null);
+    }
+
     SetDimming(enable) {
         this._manager.dimming = enable;
     }
-- 
2.39.5